Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PP-12295 Add new Stripe terms and conditions page #4182

Merged
merged 1 commit into from
Feb 27, 2024

Conversation

iqbalgds
Copy link
Contributor

  • This page will contain the latest Stripe terms and conditons.
  • This page will be on a brand new URL: /policy/stripe-terms-and-conditions
  • Intend to put this live but hidden from real users.
    • This will allow the team to review the content and request updates.
    • We expect there to be some late edits as we are working with the legal team to finalise the content.
  • This page is implemented differently from the other /policy/ pages:
    • The other pages generate a 1 hour link to a policy PDF file and follow a specific page layout.
    • This new page will look different and we need to release it quickly.
    • Making this work with the existing code would have required too much effort.
    • Therefore, this was implemented differently as a static page - which is quicker to implement without affecting the existing policy pages.

- This page will contain the latest Stripe terms and
  conditons.
- This page will be on a brand new URL:
  /policy/stripe-terms-and-conditions
- Intend to put this live but hidden from real users.
  - This will allow the team to review the content and request updates.
  - We expect there to be some late edits as we are working with the
    legal team to finalise the content.
- This page is implemented differently from the other /policy/ pages:
  - The other pages generate a 1 hour link to a policy PDF file and
    follow a specific page layout.
  - This new page will look different and we need to release it quickly.
  - Making this work with the existing code would have required too much
    effort.
  - Therefore, this was implemented differently as a static page - which
    is quicker to implement without affecting the existing policy pages.
Copy link
Contributor

@JFSGDS JFSGDS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks OK to me.

@iqbalgds iqbalgds merged commit 80990d6 into master Feb 27, 2024
12 checks passed
@iqbalgds iqbalgds deleted the pp-12295-add-new-stripe-tandc-page branch February 27, 2024 10:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants